|
|
스크린샷 2022-03-30 오전 8.56.34.png (151,552 bytes) |
|
|
Hi River, I want to remind you for this ticket. Thanks. |
|
|
Hi Wooshin, Thanks for your remind. For those API, I have some questions: 1.DISPLAY_getHDCPVersion: Does DISPLAY_getHDCPVersion() mean to get the current TV supported HDCP version? Or DISPLAY_getHDCPVersion() mean to get the current authenticated HDCP version? 2.DISPLAY_getHDCPState: Does DISPLAY_getHDCPState() will reflect the real authentication status or just what DISPLAY_setHDCPEnable() had configured? 3.DISPLAY_getColorSpace: Does DISPLAY_getColorSpace() mean to get the current TV supported Color space of current resolution? Or DISPLAY_getColorSpace() mean to get the current configured Color space of current resolution? 4.DISPLAY_getBitDepth: Does DISPLAY_getBitDepth() mean to get the current TV supported bit depth of current resolution? Or DISPLAY_getBitDepth() mean to get the current configured bit depth of current resolution? |
|
|
1.DISPLAY_getHDCPVersion: It should be returned set value by DISPLAY_setHDCPVersion () 2.DISPLAY_getHDCPState: It should be returned set value by DISPLAY_setHDCPEnable() 3.DISPLAY_getColorSpace: It should be returned set value by DISPLAY_setColorSpace() 4.DISPLAY_getBitDepth: It should be returned set value by DISPLAY_setBitDepth() |
|
|
Hi Wooshin, So, if get func is called before set func, then get func will return fail -1. Is it ok? |
|
|
Hi River, When set func is not called, return current value if you have any solution to read current status. Thanks. |
|
|
Hi River, Please fill below function also. HRESULT AMP_SND_SetDelay(AMP_IN AMP_SND_PATH Path, AMP_APP_PARADELAY *stDelay); HRESULT AMP_SND_GetDelay(AMP_IN AMP_SND_PATH Path, AMP_OUT AMP_APP_PARADELAY *stDelay); You can find them in BTF_API_LEVEL_6-bg5ct-ua300 branch of vendor/synaptics/btv-hal-impl/synaptics. Thanks. |
|
|
Hi Wooshin, For get func, I am confused how to implement DISPLAY_getHDCPVersion() by rule 1 "It should be returned set value by DISPLAY_setHDCPVersion ()" and rule 2 "When set func is not called, return current value if you have any solution to read current status." For exmpale, when I connect to a HDCP1.x only TV: 1.API called setHDCPVersion() with HDCP2.2, But TV only support HDCP1.X, The real HDCP status is HDCP1.x. But for rule 1 "It should be returned the set value by DISPLAY_setHDCPVersion ()", I should return HDCP2.2 (by store what setHDCPVersion() was set.) when getHDCPVersion() is called. 2.If no setHDCPVersion() is not set yet, I get the real HDCP status as HDCP1.x and return HDCP1.x when getHDCPVersion() is called. For these two cases, it returns different result for DISPLAY_getHDCPVersion(). Do this is what you want? |
|
|
Hi river, I have discussed with M/W ( SPTEK ) 1. All get functions is able to be called before set functions. In this case, get function should return current set value on system. 2. In case of HDCP version, error value should be returned if not supported version. 3. Set function is fail, error code should be returned. And then get function should be return current set value. Thanks |
|
|
Hi Wooshin, OK, I got it. I will try to implement it as what you described. For new API, "int AMIXER_SetAVOffset(AMIXER_HANDLE amixer, int offset)" and "int AMIXER_GetAVOffset(AMIXER_HANDLE amixer, int *offset)", do yo want me to use AMP_SND_SetDelay() and AMP_SND_GetDelay() to implement it? Is it suggested by Synaptics? The AMP_SND_SetDelay() has to specified a PATH as first parameter. It usually is AMP_SND_PATH_71 for Speaker. But do I have to consider HDMI passthrough mode status and config HDMI delay instead of Speaker in passthrough mode being enabled case? |
|
|
Hi River, Yes, you use iAMP_SND_SetDelay for these API. It will be used for A/V synchronization of HDMI when audio out is set passthrough. Thanks |
|
|
Hi Wooshin, One more question: If DISPLAY_setColorSpace() is set with AVP_COLOR_SPACE_AUTO or DISPLAY_setBitDepth() is set with AVP_BIT_DEPTH_AUTO, then should DISPLAY_getColorSpace() and DISPLAY_getBitDepth() returns AVP_COLOR_SPACE_AUTO /AVP_BIT_DEPTH_AUTO or the real colorspace/bitdepth which is selected by AUTO? |
|
|
Hi River, After set func, should be returned AVP_COLOR_SPACE_AUTO otherwise real value should be returned. Thanks. |
|
|
Hi Wooshin, The current implement of DISPLAY_SetResolution(DISPLAY_HANDLE display, BTF_DISPLAY_Resolution resolution) will also choose the best color space and bit depth for resolution parameter (because Synaptics needs these three information to set resolution). So, if DISPLAY_SetResolution(() is called first, I will also return AVP_COLOR_SPACE_AUTO for DISPLAY_getColorSpace() and AVP_BIT_DEPTH_AUTO for DISPLAY_getBitDepth(). Is it ok for you? |
|
|
Hi River, Actually, It is relative with UI menu setting so, it will be tested with UI menu operation. Because ui menu is not ready currently, do implement as your idea and then let's you debug when ui is ready. Thanks. |
|
|
Hi Wooshin, I commit the following Display APIs to bitbucket branch "soc-syna/BTF_API_LEVEL_6/bg5ct". I think SPtek can try them. If any problem found or need to be modified, then we can discuss the details. 1. DISPLAY_setColorSpace/DISPLAY_getColorSpace 2. DISPLAY_setBitDepth/DISPLAY_getBitDepth 3. DISPLAY_setHDCPEnable/DISPLAY_getHDCPState 4. DISPLAY_setHDCPVersion/DISPLAY_getHDCPVersion commit 0a38ab3bc92e65f6ca95bd5b04e4b49db7b0eb5c (HEAD, soc-syna/BTF_API_LEVEL_6/bg5ct) Author: river wong <river.wong@fii-foxconn.com> Date: Mon Apr 18 15:18:41 2022 +0800 [BTVSBOX-608] Implement Display APIs defined on BTF_API_LEVEL_6: 1. DISPLAY_setColorSpace/DISPLAY_getColorSpace 2. DISPLAY_setBitDepth/DISPLAY_getBitDepth 3. DISPLAY_setHDCPEnable/DISPLAY_getHDCPState 4. DISPLAY_setHDCPVersion/DISPLAY_getHDCPVersion I will keep implementing BTF Level 6 AV Delay API. If it is finished, I will let you know. |
|
|
스크린샷 2022-04-19 09.46.30.png (72,219 bytes) |
|
|
Hi Wooshin, Could you confirm with SPtek if there are any other APIs or properties that needs to be implemented for BTF Level 6? Could they give all of it at one time? Then we can discuss these requirements otherwise we may need to change the previous implemented APIs because the new added requirement. |
|
|
Hi River, Owner of BTF is SKB and they are adding the requirement for BTF LEVEL 6. When i check with them, they said that there is no exact specification. I am sorry but there is possibility to add new requirement. |
|
|
Hi Wooshin, I have one question about the properties. Except EDID, there are already BTF APIs to get HDCP/Resolution/ColorSpace/ColorDepth, Why they still need these properties? |
|
|
Hi River, It will be used in log capture system of SKB. Thanks. |
|
|
Hi Wooshin, It seems all the properties can be implemented in Synaptics SDK because any set APIs should use Synaptics SDK to implement the function. And other products like AI2 also needs these properties. We recommend these properties to be imlpemented by Synaptics. Then all products which use Synaptics solution can all has these properties support. |
|
|
Hi River, I will contact with skb manager to discuss your recommendation. Thanks. |
|
|
Hi River, SKB already did contact Synaptics and got a not good response from them. Synaptics is saying these function should by implemented by product owner(manufacturing company). Thanks. |
|
|
Hi Wooshin, AudioDelay APIs are committed to branch BTF_API_LEVEL_6-bg5ct-ua300. Please note, we only verify with YouTube video about the audio delay for 100ms and 200ms and result is OK. But for IPTV, we do not have av sync test video for IPTV, so we can not verify it. Please let us know if you find any issue about these APIs. commit 5f33b8096103344f171c1626c8257cb7c558d951 (HEAD, soc-syna/BTF_API_LEVEL_6-bg5ct-ua300) Author: river wong <river.wong@fii-foxconn.com> Date: Wed Apr 27 14:01:17 2022 +0800 [BTF-595] Implement : AudioDelay API |
|
|
Hi Wooshin, For EDID property, if HDMI cable is not connected, what String should I show? |
|
|
Hi River, You can use "NG". Thanks. |
|
|
Hi River, When is target date to finish it ?, Thanks. |
|
|
Hi Wooshin, For some properties, I am still studying how to implement it (especially HDCP and EDID). Could I know when the bitbucket branch for BTF level 6 will be created? (Because I think maybe I need to change code in syna-release/ampsdk for some properties.) |
|
|
Hi River, SPETEK engineers are off until this week. I can check it at next Monday. Thanks. |
|
|
Hi River, I did contact SPTEK. They want you to make branch for your changing and then you inform to us about branch. And then you can change something in new branch. Thanks. |
|
|
Hi Wooshin, Thanks. I found a way to implement the properties in BTF module only. So, it will not need to change code in Synaptics APIs. I am now testing it, if no problem. I will commit the code today or tomorrow. I will inform you after I commit it to bitbucket. |
|
|
Hi Wooshin, I test all the properties except EDID NG case. Becaseu we do not TV with bad EDID. But I think if Synaptics API is implemeted correctly, it should be no problem for EDID NG case. So, I committed the properties implementation to bitbucket. commit 28d7f596ad88aeeba9cc75fdba58db04fc3f47cd (HEAD, soc-syna/BTF_API_LEVEL_6/bg5ct) Author: river wong <river.wong@fii-foxconn.com> Date: Mon May 9 14:02:44 2022 +0800 [BTVSBOX-608] Add properties for HDMI. EDID: vendor.skb.hdmi.edid HDCP: vendor.skb.hdmi.hdcp Resolution: vendor.skb.hdmi.size Color Space: vendor.skb.hdmi.cs Color Depth: vendor.skb.hdmi.bit |
|
|
All requested APIs and Properties are implemented. |
- yk.jung ( (ALTech) Younkwang Jung ) developer
